Skip to content

Conversation

@scaliby
Copy link
Member

@scaliby scaliby commented Oct 29, 2025

Description

Workloads are already tested here and here for pathways, so there is no need to test this twice.

Issue

b/445869176

Testing

N/A

@scaliby scaliby marked this pull request as ready for review October 29, 2025 16:52
@scaliby scaliby changed the title build: remove duplicated workload tests Remove duplicated workload tests Oct 29, 2025
@scaliby scaliby requested a review from jamOne- October 30, 2025 08:08
Copy link
Collaborator

@FIoannides FIoannides left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These two seem to be missing, http://screen/6uVbpEc8Y4qZoVf from the set that we are keeping, can we add them?

@scaliby scaliby force-pushed the remove-workload-tests branch from f76d448 to 4f3814f Compare October 30, 2025 10:59
@scaliby
Copy link
Member Author

scaliby commented Oct 30, 2025

These two seem to be missing, http://screen/6uVbpEc8Y4qZoVf from the set that we are keeping, can we add them?

Added. Should be good now.

@scaliby scaliby requested a review from FIoannides October 30, 2025 10:59
@scaliby scaliby marked this pull request as draft October 30, 2025 11:00
@scaliby scaliby force-pushed the remove-workload-tests branch from 4f3814f to e697c6e Compare October 30, 2025 11:01
@scaliby scaliby marked this pull request as ready for review October 30, 2025 11:01
@scaliby scaliby requested a review from FIoannides October 30, 2025 13:39
@scaliby
Copy link
Member Author

scaliby commented Oct 31, 2025

@jamOne- could you PTAL? This is blocked as you requested changes.

Copy link
Collaborator

@jamOne- jamOne- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry

@scaliby scaliby enabled auto-merge (squash) October 31, 2025 10:03
@scaliby scaliby merged commit 9bc4fd5 into AI-Hypercomputer:main Oct 31, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants